-
Notifications
You must be signed in to change notification settings - Fork 101
feat: Enable ALTS hard bound token in Bigtable w/ direct access #2695
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
2cf12e6 to
90e032f
Compare
90e032f to
c296c92
Compare
| .setAttemptDirectPath(true) | ||
| // Try to fetch a hard-bound access token for direct access if the runtime | ||
| // environment supports it. | ||
| .setAllowHardBoundTokenTypes( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can set it as default. Not a hard requirement, but could we add a env var to disable this incase we have any problem? Better to use env var rather than rollback client to prev version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a great point. Done. Thanks for the tip!
|
@sushanb Thanks for approving the PR. I doubt the CI failure was due to the change here. Could you help rerun it? |
|
/gcbrun |
No description provided.